Skip to content

MB-71383: Define logging functions for GPU - #92

Closed
capemox wants to merge 2 commits into
masterfrom
gpu-logging
Closed

MB-71383: Define logging functions for GPU#92
capemox wants to merge 2 commits into
masterfrom
gpu-logging

Conversation

@capemox

@capemox capemox commented Apr 13, 2026

Copy link
Copy Markdown
Member

Defines two new callbacks for GPU related errors:

  1. GPUToCPUCloneErrorCallback: called when the clone to GPU fails
  2. GPUErrorCallback: called when other generic GPU errors occur

Comment thread vector.go

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new (exported) callback types and associated string keys intended to let callers hook logging/handling for GPU-related vector search errors.

Changes:

  • Introduces GPUToCPUCloneErrorCallback and GPUErrorCallback function types.
  • Adds two new internal/config keys for registering those callbacks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread vector.go Outdated
Comment thread vector.go Outdated
@capemox
capemox marked this pull request as ready for review April 14, 2026 04:34
@CascadingRadium

Copy link
Copy Markdown
Member

As mentioned in my zapx comment, please avoid doing this segment config usage for stats collection.

@capemox capemox closed this Apr 17, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Done in GPU-Accelerated Vector Search Apr 17, 2026
@capemox
capemox deleted the gpu-logging branch April 17, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants